Skip to content

Add initial operation for LIN bus #73

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 6, 2023
Merged

Add initial operation for LIN bus #73

merged 3 commits into from
Oct 6, 2023

Conversation

@bmenne-dspace
Copy link
Contributor

@bmenne-dspace bmenne-dspace commented Sep 21, 2023

Add initial operation for LIN bus

@bmenne-dspace bmenne-dspace linked an issue Sep 21, 2023 that may be closed by this pull request
Add initial operations for LIN (low-cut) #68
Closed
@bmenne-dspace bmenne-dspace added the lin Issue related to LIN bus label Sep 21, 2023
@bmenne-dspace bmenne-dspace added this to the 1.1.0-alpha milestone Sep 21, 2023
@bmenne-dspace bmenne-dspace marked this pull request as ready for review September 22, 2023 11:09
@bmenne-dspace
Copy link
Contributor Author

bmenne-dspace commented Sep 22, 2023

@andreas-junghanns this is our first proposal for LIN operations. Maybe we should just merge it?! We will write additional content (explaining chapters) like within the CAN part and can verify it if the specification for LIN is complete.

@bmenne-dspace bmenne-dspace self-assigned this Sep 22, 2023
@chrbertsch
Copy link
Collaborator

chrbertsch commented Sep 26, 2023

Design Webmeeting: Let's merge it and improve it later.

@chrbertsch
Copy link
Collaborator

chrbertsch commented Sep 26, 2023 •
edited
Loading

Should we merge this into main or into a different 1.1.0 branch? (@pmai , @andreas-junghanns )
We do not plan to release this in the 1.0.0 release ...

@bmenne-dspace
Copy link
Contributor Author

bmenne-dspace commented Oct 6, 2023

Should we merge this into main or into a different 1.1.0 branch? (@pmai , @andreas-junghanns ) We do not plan to release this in the 1.0.0 release ...

@chrbertsch I think we can merge it. During the F2F meeting in PB, @pmai 's idea was to remove the remaining chapters of the other buses for a 1.0.0 release and thus develop everything in a consolidated manner. We have to remove the (currently empty) chapters for LIN/FlexRay/Ethernet for 1.0.0 anyway.

@bmenne-dspace bmenne-dspace merged commit fdc7e80 into main Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrbertsch chrbertsch chrbertsch approved these changes

@andreas-junghanns andreas-junghanns Awaiting requested review from andreas-junghanns

@pmai pmai Awaiting requested review from pmai

Assignees

@bmenne-dspace bmenne-dspace

Labels

lin Issue related to LIN bus

Projects

None yet

Milestone

1.3.0-alpha

Development

Successfully merging this pull request may close these issues.

Add initial operations for LIN (low-cut)

3 participants

@bmenne-dspace @chrbertsch